Filling as a good first bug to learn workflows.

aKeyIDs.size()
should use .empty() instead

https://searchfox.org/mozilla-central/rev/fa69d8b248e6c1df670aa6b019e30ec37e6672be/media/gmp-clearkey/0.1/ClearKeyUtils.cpp#265

As the change is trivial, it is just to learn how to contribute to Firefox.

Found by http://clang.llvm.org/extra/clang-tidy/checks/readability-container-size-empty.html

Tutorials to contribute:
https://firefox-source-docs.mozilla.org/tools/docs/contribute/how_to_contribute_firefox.html
https://firefox-source-docs.mozilla.org/contributing/stack_quickref.html

Please don't ask for the bug to be assigned. It will be automatically assigned to the first patch.

In ClearKeyUtils.cpp: the 'empty' method should be used to check
for emptiness instead of 'size'

Just to remember how to contribute to Mozilla :)

@Sylvestre could you please recommend some new bug to continue?:)

https://hg.mozilla.org/mozilla-central/rev/caf28310076d

Timur, interested to run:
https://bugzilla.mozilla.org/show_bug.cgi?id=1617532
on the whole tree?

Yes. Going to take care about it.